Skip to content

Conversation

@0xfabio
Copy link

@0xfabio 0xfabio commented Oct 23, 2025

We are running into the already opened issue, so I thought I could quickly fix this.

#652

Testing

Running the following command fails with the current main, but works as expected in with this fix:

helm template charts/cluster --set "recovery.pgBaseBackup.secret.name=foo"

@0xfabio 0xfabio requested a review from itay-grudev as a code owner October 23, 2025 15:28
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. bug Something isn't working chart( cluster ) Related to the cluster chart labels Oct 23, 2025
@0xfabio 0xfabio force-pushed the 652-fix-pg-basebackup-secret-mismatch branch from 66c1389 to 122d12b Compare October 23, 2025 15:54
@0xfabio 0xfabio force-pushed the 652-fix-pg-basebackup-secret-mismatch branch from 122d12b to e84745b Compare October 23, 2025 15:58
@0xfabio
Copy link
Author

0xfabio commented Oct 30, 2025

Hi @itay-grudev, any chance we could get a review here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working chart( cluster ) Related to the cluster chart size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant